Skip to content

Migrate to plumcp#355

Merged
ericdallo merged 8 commits intomasterfrom
plumcp
Mar 12, 2026
Merged

Migrate to plumcp#355
ericdallo merged 8 commits intomasterfrom
plumcp

Conversation

@ericdallo
Copy link
Member

Fixes #316

ericdallo and others added 5 commits March 11, 2026 18:29
plumcp's default list_changed notification handlers (fetch-prompts,
fetch-resources, fetch-tools) fire immediately when the server sends
notifications before the initialize handshake completes. Since MCP
servers ignore requests before initialize, these handlers block forever
waiting for responses that never come, preventing the client from
completing initialization.

Override the three list_changed handlers with guarded versions that
only dispatch after initialize-and-notify! succeeds.

🤖 Generated with [eca](https://eca.dev)

Co-Authored-By: eca <noreply@eca.dev>
@ericdallo ericdallo force-pushed the plumcp branch 2 times, most recently from ace6a04 to 54bd1fd Compare March 12, 2026 17:58
- Remove flushing-stdio-transport wrapper (flush bug fixed in beta3)
- Replace manual initialized? atom with wrap-initialized-check

🤖 Generated with [eca](https://eca.dev)

Co-Authored-By: eca <noreply@eca.dev>
@ericdallo ericdallo merged commit 70ef8a3 into master Mar 12, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Integrate with plumcp for MCP functionality

1 participant